Ignore errors during lconvert

Starting with qt5 5.15.4 lconvert errors with

QM-Format error

for some calls and thus thus fails the build.

Gbp-Pq: Name 0009-Ignore-errors-during-lconvert.patch

Add workaround for failing test on i386

Gbp-Pq: Name 0007-Add-workaround-for-failing-test-on-i386.patch

Disable-donation-nagging

Gbp-Pq: Name Disable-donation-nagging

htmldoc-location

Gbp-Pq: Name htmldoc-location

gpsbabelfe_do-not-check-for-newer-version

Disable check for newer versions on start.

Gbp-Pq: Name gpsbabelfe_do-not-check-for-newer-version

gpsbabelfe_opt-in-for-usage-data-reporting

Make statistics reporting an opt-in option.

Gbp-Pq: Name gpsbabelfe_opt-in-for-usage-data-reporting

gpsbabelfe_disable_version_check

Disable version mismatch check.

Gbp-Pq: Name gpsbabelfe_disable_version_check

Update changelog for 1.9.0+ds-3 release

Depend on locales-all for tests

Update changelog for 1.9.0+ds-2 release

Fix build on 32bit archs

Fix man page generation

Update changelog for 1.9.0+ds-1 release

Update d/clean

Closes: #1044858

Switch build system to cmake

Rebase patches

Update d/copyright

Update upstream source from tag 'upstream/1.9.0+ds'

Update to upstream version '1.9.0+ds'
with Debian dir 3302622e15137cc768eeb3cfe6c51b2f1d640e8b

New upstream version 1.9.0+ds

rc1 for release 1.9.0 (#1186)

archive docs in CI (#1187)

* create document generation workflow

* fix webdocdir bug

* update checkout

* update document for cmake variable renames.

bump qt build from 6.5.2 to 6.5.3 (#1185)

Remove final remnants of 'exported' (#1146)

* Remove final remnants of 'exported'

* incorporate review comments in duplicate filter,

and remove deleted exported keyword from unicsv doc.

---------

Co-authored-by: Robert Lipe <robertlipe@gmail.com>

tidy fixes (#1179)

* tidy readability-convert-member-functions-to-static

* tidy readability-make-member-function-const

* kill trailing space

fix various "Redundancies in code"  (#1178)

* fix "Redundant elaborted type specifier"

* fix "Redundant 'inline' specifier"

* fix "redundant 'const' specifier on a constexpr variable"

* remove "Redundant qualifier"

* add /out/, /.vs/ to .gitignore, and sort.

fix SchemaArrayField/SchemaArrayData mismatch (#1177)

* fix SchemaArrayField/SchemaArrayData mismatch

related to IGC additions from #1060 and #1054.

* retire global trait collection

* use class member to store track traits instead of fs data.

* fiddle with kml accumlate hash

* compute number of wp_fields

* correct bug concerning track specific traits.

* simplify multi-track array decisions.

* simplify kml multi-track schema decisions.

Integrate OAT and SIU with Waypoints (#1060)

* Integrate OAT and SIU with Waypoints

* Type corrections, convert set_value() overload to nullptr* check

* #1062 Fix core dump when merging tracks with disparate IGC extensions

* Reduce `value` variable scope when writing SimpleArrays

* Debug oopsie bugfix, fix core dump when merging IGC files with no extensions

* Collapse redundant if statement

* Fix brain fart

* Eliminate igc_fsdata::has_value()

* Fix subtle satellite count bug

* Fix small sat count bug, and eliminate some ifdefs

* Make some IGC exts override default WP members

* Include IGC line-by-line debug info

* Remove accidentally included file

* delete obsolete macros

---------

Co-authored-by: tsteven4 <13596209+tsteven4@users.noreply.github.com>

fix noreturn booboos. (#1176)

* fix warning: function declared ‘noreturn’ has a ‘return’ statement

* geez, another booboo using noreturn.

Merge pull request #1175 from tsteven4/refact

Apply new resharper refactoring "Redundant dereferencing and taking address"

fix reserved-macro-identifier clang diagnostic

use noreturn attribute

drop extra semicolons

don't use posix but non-iso argument reordering in print specifiers.

remove obsolete declarations

Apply new resharper refactoring "Redundant dereferencing and taking address"

make all but the stack filter dynamic. (#1174)

Bump actions/checkout from 3 to 4 (#1172)

Bumps [actions/checkout](https://github.com/actions/checkout) from 3 to 4.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](https://github.com/actions/checkout/compare/v3...v4)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: tsteven4 <13596209+tsteven4@users.noreply.github.com>

mkshort becomes a class (#1173)

* make mkshort a class

* add new file mkshort.h

* refactor names

* use bools with mkshort setters

convert mkshort from char* to QByteArray (#1171)

* convert mkshort from char* to QByteArray.

and store conflict count directly in hash.

* fix potential detach

and rename vars

* tweak whitespace removal.

* don't split codepoints that comprise a grapheme.

* enhance mkshort test

* fix enhanced test

* test mkshort replacements

* remformat mkshort with astyle

* add mkshort test to cmake

Update gpsbabel_man.xml (#1170)

Without a dash this will resemble the "tar" command!

fix mkshort unique (#1168)

* fix mkshort unique.

1. we were off by one when deciding if we could just concatentate
the generated suffix to the name, failing to use the last available
character.
2. when attempting to copy the generated suffix to a presumably
shortened name, and the generated suffix was longer than the name,
the target of the copy was outside the name buffer.  In the common
case that the rank of size_t was greater than the rank of int, the
target was well beyond the end of name, not before as one would expect
with signed arithmetic.

With the new algorithm when the target length is insufficient to
fit both the name and the suffix we will only truncate the name as
required to fit the truncated name and the complete suffix in the
target length(as opposed to the original length of the name).
We fatal if this is not possible.

Add a test case to exercise the make unique code.

* fix reference mode.

* add notes for future enhancements

* fix testcase cut and paste booboo

* refactor mkshort input from char* to QByteArray.

convert googletakeout format to dynamic. (#1167)

delete obsolte and unused macro. (#1166)

delete undefined declaration. (#1165)

The declaration appeared in 6a9a05864, 7/25/2011, without a definition
or usage.

fix memory leak in mkshort from d7dfed6, 1/23/2005. (#1162)

Add support for Google Takeout Location History (#1160)

* Add support for Google Takeout Location History

This adds support for parsing the location history provided by Google Takeout.
This makes it possible to convert a month's, year's, or your entire location
history at once instead of having to export one day's history at a time on
Google Maps.

There is documentation in the `xmldoc` folder with some examples.

It works great, but there are one, possibly two, other additions I'd like to
make in following diffs:

1. Add the ability to select date ranges. For example, I went on a road trip
   that started in the last week of February, but right now I can only select
 tracks at the resolution of a whole month.

2. Add support for the higher-resolution "roadSegment" blocks in newer
   "activitySegment" blocks. This will have to be optional as you have to
 query the Google Maps API to get lat/long, which costs money. However,
 if you're willing to pay for it, you can get great detail out of it.
 They look like this:

```
       "roadSegment": [{
          "placeId": "ChIJBdXmuuWaK4gRoiClERhZcy8",
          "duration": "76s"
        }, {
          "placeId": "ChIJ4ekNlOWaK4gRxu-moMRFo60",
          "duration": "58s"
        }, {
          "placeId": "ChIJ10qEheWaK4gRQ1T24wEuuN4",
          "duration": "58s"
        }, {
          "placeId": "ChIJe7NWfu-aK4gRFHBB6P0PMe4",
          "duration": "58s"
[...]
```

* rename variable

* this TODO is done

* now builds on focal, fix documentation

* patch -p2 < ~/testo.log

* use `&&` instead of `and` for old compilers

* don't leak objects when we skip events at exact lat/lon 0/0

* use a reference (even though the compiler would have optimized that away)

* rename to googletakeout

* fix docs

* Address code review requests

* add `googletakeout.h` to HEADERS
* convert most `#define`s to static members
* don't pollute global namespace
* rename logging functions
* convert some methods to static functions for clang-tidy
* `title_case` loop by reference instead of by index (clang-tidy)
* better use of debugging levels
* drop dead simplifiedRawPath code for now
* drop empty tracks
* default constructor for GoogleTakeoutInputStream
* add some tests that check the content of the data

Also, I found some unofficial documentation about the location history
format and added that (thanks @CarlosBergillos !)

* apply @tsteven4 's patch. thank you!!

* add license

* fix logging

* As I assigned the correct license to my code, I am skeptical that this makes a difference, but give Robert co-copyright.

* make the test more boring

* rm xml_slice

* revert constructor change from @tsteven4 's patch

* apply @tsteven4 's latest constructor

* apply @tsteven4 's latest patch -- `cmake --build . --target check` passes

---------

Co-authored-by: tsteven4 <13596209+tsteven4@users.noreply.github.com>

retire strupper, strlower. (#1164)

kill tpo2 write code, which has been dead since 46676d105, 5/17/2006. (#1163)

introduce clang-tidy to CI and integrate with codacy (#1071)

* add codacy clang-tidy to workflows

* attempt to run clang-tidy on ci

* update jammy image.

get rid of ppa
add clang-tidy, jq.

* fixes.

* whittle down tidy checks.

* kick out another tidy warning

resolve maintanence issue in interpolate & resample filters. (#1161)

* resolve maintanence issue in interpolate filter.

Instead of manually creating a deep copy of the route list with
an empty waypoint list we operate on the original route list by
swapping it's waypoint list with an empty list.  This is both more
efficient and easier to maintain.

* resolve maintanence issues in resample filter.

Instead of manually creating a deep copy of the route list with
an empty waypoint list we operate on the original route list by
swapping it's waypoint list with an empty list.  This is both more
efficient and easier to maintain.

For efficient deletion use track_del_marked_wpts.

remove obsolete files (#1158)

* remove obsolete files

* delete some more obsolete files

adjust greatcircle conversion factors, kill gcgeodist cheat. (#1156)

* tweak greatcircle conversions.

* modify arc-project test to use microdegrees instead of nanodegrees.

* kill unused metric prefix conversion macros.

remove erroneous addition from #1151. (#1157)

The need for this statement was eliminated with the introduction of
wpt_flags.marked_for_deletion.

use efficient methods to delete trkpts in track filter. (#1155)

* use efficient methods to delete trkpts in track filter.

* clean up some code issues flagged by windows tools.

fix interator invalidation issue with trackfilter segment option. (#1153)

and add a test case that demonstrates the issue.

use std::prev instead of --c.end() (#1154)

--c.end() is not guaranteed to compile.

Introduce efficient member functions for deleting waypoints (#1151)

* Introduce efficient member functions for deleting waypoints

New functions with complexity O(n) for deleting pre-marked
waypoints are:
  WaypointList::del_wpts
  RouteList::del_wpts
  del_wpts
  route_del_wpts
  track_del_wpts

Use those functions instead of the inefficient versions
WaypointList::waypt_del
RouteList::del_wpt
waypt_del
route_del_wpt
track_del_wpt.
When these functions are using while looping over a waypoint list
the overall complexity is O(n^2).  This is because these functions
themselves amortize to O(n).

* review nits.

* tweak deletes

* use dedicated wpt deletion flag.

Reduce complexity of simplify filter (#1149)

* fix bugs with simplify filter

1. With the length option the last point deleted took the total error
over the specified limit.

2. When computing the total error if another point is deleted it was
possible to refer to an xte record that needed to be updated due to
the deletion of one of its neighbors.

* wip on reduced complexity simplify filter.

* mimic ineheitence of new_trkseg flag in simplify filter.

* cleanup simplify header.

* work around apparent Qt5 QMap bug.

* fix shadowing issues.

* astyle

* correct spelling.

* kill Waypoint route_priority.

time read, filter and writer operations (#1150)

* time read, filter and writer operations

when using -D > 0.

* take stray dog to the pound.

fix bugs with simplify filter (#1148)

1. With the length option the last point deleted took the total error
over the specified limit.

2. When computing the total error if another point is deleted it was
possible to refer to an xte record that needed to be updated due to
the deletion of one of its neighbors.

Update changelog for 1.8.0+ds-7 release

Fix package short description

Closes: #1042359

Update changelog for 1.8.0+ds-6 release

Move package description to source package

Bump policy version (no changes)

Don't use qtwebengine5-dev on mipsel, package to be removed

Closes: #1041912

improve correctness and speed of duplicate filter. (#1144)

* improve correctness and speed of duplicate filter.

The duplicate filter could errouneously delete points that were
not duplicates if the crc's happened to match.

waypt_del(Waypoint*) is inefficent as it requires a search of the
list to find the matching waypoint.  Support waypt_del with iterators.

* retire util_crc.cc

* improve duplicate to linear complexity

* polish new list creation.

* Remove final remnants of 'exported'

* Revert "Remove final remnants of 'exported'"

This reverts commit 6996e4d7dd4b1812f922edd2c5bd9a3d1bc6fada.

---------

Co-authored-by: Robert Lipe <robertlipe@gmail.com>

update Qt 6.5.1 -> 6.5.2 in CI. (#1140)

sync test case list in CMakeLists.txt with testo.d. (#1143)

Yes, manual syncing is suboptimal.

fix mkshort qHash return type. (#1142)

reduce complexity of gdb waypt searchs from O(n^2) to O(n) (#1141)

* use QHash for gdb waypoint searches.

This reduces the complexity from O(n^2) to O(n).

For the reader two parallel hashes are maintained with different
keys but identical values.  This is necessary because sometimes
we want to match the name and position, and other times we just want
to match the name.

* workaround missing qHashMulti in Qt5.

and supply missing default for our qHash functions.

* fix MSVC C2666 errors due to an erroneous assumption.

In Qt6 the return type of qHash, and the type of the seed,
happens to be the same as QHash::size_type.
However, this isn't true in Qt5.

reduce complexity of mkshort from O(n^2) to O(n). (#1139)

* reduce complexity of mkshort from O(n^2) to O(n).

* add converting constructor for mkshort hash key

Deprecate navilink, sbn, sbp. (#1138)

As discussed in
https://github.com/GPSBabel/gpsbabel/pull/1090
these units have long since been deprecated by their maker. We have no
known remaining users. Numerous attempts (including  phone calls and
attempt to contact via the windsurfing community) to reach the last
known users have all failed.

Buh Bye.

deprecate wbt formats. (#1136)

eliminate ppa for qlibraryinfo bug (#1135)

Strptime update (#1133)

* prepare to update strptime

* update strptime to glibc 2.37, with modifications.

* add missing cmake file.

* quiet MSVC warnings with strptime

* quiet codacy complaints about strptime

third party code

* use _strnicmp on windows

* ensure strptime works with signed enums for windows.

fix clang 15 warnings on windows (#1132)

* fix some format specifier issues

that came to light trying to get clang to work under visual studio.

* fix another windows only warning

* fix some more wordsize issues.

add gnu attribute to check functions with printf like parameters. (#1129)

and fix a bug revealed by the warning.

fix gdb debug (#1128)

* fix gdb dbg.

* use bool literals

fix humminbird icon matching, (#1126)

* fix humminbird icon matching,

eliminating one use of xasprintf.
This was broken in 6029c8267 over a decade ago.

* eliminate unnecessary xasprinf usage.

* eliminate another xasprintf usage.

* eliminate xasprintf usage.

* retire xasprintf, xvasprintf.

drop support for filesystem access via garmin usb. (#1127)

add fedora 38 to CI (#1125)

fix nits with trackfilter (#1124)

* fix nits with trackfilter

* deprecate mapconverter readme.

mapconverter style file was removed in #870

* catch reference files up

switch some builds to Qt 6.5.1 (#1119)

* update some builds to Qt 6.5.1

* conditional code for QVersionNumber evolution

fix garmin serial/usb device character encoding/decoding. (#1117)

* attempt to use correct codec with garmin reader.

the garmin writer is unchanged as of yet.

* fix garmin writer wrt encoding.

* add garmin option to force codec.

* update serialization ref files for garmin codec option.

* fix potential overwrite bug with route/track names

in garmin writer.

* fix up for old Qt.

* correct assertion

* use static_assert to for fixed buffer size checks.

* document garmin codec option

* don't clean rtept/trkpt names for newer devices.

We clear badchars for newer devices already, effectively not
cleaning wpt names.

refactor unicsv & xcsv date time handling (#1114)

* refactor unicsv date time handling.

* restrict utc option range to match Qt offsetfromutc

* move xcsv from C-style legacy time to Qt.

* fill in lower order date/time fields when parsing.

* don't return date/time if we don't have one!

* enhance xcsv date/time testing, fix bug.

* csv format date time adjustements.

use QDateTime::fromString to parse iso date times.
return invalid QTime from addhms if parsing fails entirely.

* datetime display fixes.

For the xcsv writer:
avoid priting date/time fields when there isn't a valid date/time.
when printing with am/pm use times from a 12 hour clock.
add support for printing dotnet time.
use QDateTime::toString to print ISO time.
For the xcsv and unicsv readers:
use QDateTime::fromString instead of xml_parse_time when reading
ISO datetimes.  This avoids xml_parse_times intentional odd behavior
of treating non-timezoned times as UTC.

add a test of the xcsv writer time related fields.  This only runs
in America/Denver time zone.  As setting the time zone is system
dependent the test only runs if tzselect is available.  If so it
assumes America/Denver is available.

correct documentation to give a sensible format for xcsv field GMT_TIME.

* add reference files for new test.

* silence xcsv reader conversion warnings on empty strings.

* warn on parse errors reading excel time.

* xcsv date/time fixes.

fix addhms to account for 12/24 hour clock.
don't print invalid datetimes with iso_time, iso_time_ms.

add testcases to exercise all the xcsv reader date/time flavors.

* clarify HMSL, HMSG wrt 12/24 hour clocks.

* add missing reference file.

* enhance xcsv for HMS[L|G] before or after [LOCAL|GMT]_TIME

* restore YYYYYMMDD to use UTC.

This has been broken for some time.  Mail from 2012 indicates the intent
was UTC (https://sourceforge.net/p/gpsbabel/mailman/message/29544538/)

* unicsv review catches

don't ripple into trouble with msec rounding.
pass outputs that may or may not be written as references.

* retire xcsv fields HMSG_TIME, HMSL_TIME.

This is potentially a user visible change.  It could require
users to rewrite any style files they have created that use
these fields.

These are replaced by repeated use of GMT_TIME and LOCAL_TIME.
This eliminates our pain over 12/24 hour clock issues. strptime/
strftime (as well as QTime) have distinct conversion specifiers
for hours with 12/24 hour clocks.  Our support for HMSG_TIME,
HMSL_TIME used integer conversion specifiers for hours, minutes
and seconds.  This made it difficult to decide if a 12 or 24 hour
clock should be used, and made it impossible to have reduced
precision values with 12 hour clock using an AM/PM designation.
We also always printed AM/PM designations.

* fix whitespace in serialization reference file

restore tab when display vector formats. (#1116)

this was lost in #1068
update reference files to be exactly what the test produces, including
exact whitespace, trailing tabs and no newline at end of file.

add testing for kml:dateTimeType. (#1112)

and fix a related bug.

Note that we incorrectly, for kml:dateTimeType, force values
without time zone qualifiers to UTC.
We do not test the optional gYearMonth time zone qualifer.
We do not test the optional gYear time zone qualifer.
We do not test the optional date time zone qualifer.

workaound curl bug in lastest windows github runner. (#1111)

update linux snap (#1110)

* switch snap from devmode to strict.

* date first in version string

* update snap: strict, no gui, layout

create a snap of the cli (#1108)

* try to build a snap on github.

* save artifact

* add test and continuous release

* permissions

* try to get sha into snap

* try harder to get sha in

* use pre-existing source

* debug missing sha

* force version into snap instead of sha

* shorten version

* make version more sortable

* avoid dying on info display

* pipes vs ors

* more work on sha

* again sha...

restrict osm timestamps. (#1107)

osm consumers may not accept fractional seconds, but all should
support whole seconds.
osm consumers may not accept time zone designations other than Z,
but all should support time zone designation Z.
See https://github.com/osmcode/osmium-tool/issues/267

datetime purity. (#1102)

eliminate some QDateTime -> gpsbabel:DateTime conversions.  These
can also help performance as DateTime defaults to UTC.

eliminate some unecessary involvment of the UNIX epoch when computing
time differences.

use QRegularExpression for xml generic tag matching. (#1099)

* use QRegularExpression for xml generic tag matching.

The kml reader now creates tracks from innerBoundryIs elements in
addition to the historic creation from outerBoundryIs elements.

* collapse kml 2.3 track elements

* const correctness improvements for xmlgeneric.

remove C style legacy time from sbn format. (#1096)

remove remnants of baroiq format. (#1095)

aka brauniger_iq.

set timespec to UTC in new skytraq code. (#1094)

I wish Qt would default to UTC instead of Local Time!

move skytraq away from C style legacy time. (#1092)

Co-authored-by: Robert Lipe <robertlipe@users.noreply.github.com>

finish removing C style legacy time from navilink. (#1093)

coverage is provided by sbp test.

move navilink towards QDateTime (#1091)

* enable navilink waypoint write test.

add rounding for round trip fidelity.
the track test has more serious mismatches.

* move navilink towards QDateTime.

reading the datalog is untested and still uses the legacy C-style
date and time library.

* use binary mode in navilink writer.